Port the proc macro attributes to the new attribute parsing infrastructure #143607
Conversation
rustbot
added
A-attributes
|
|
||
| // Not a built-in macro | ||
| None => (None, helper_attrs), | ||
| }; |
There was a problem hiding this comment.
The logic here should be equivalent to the logic above, other than that this is a bit more readable and that we use the already parsed attribute rather than parsing it from scratch
| @@ -57,7 +57,7 @@ | |||
| // see gated-link-args.rs | |||
| // see issue-43106-gating-of-macro_escape.rs for crate-level; but non crate-level is below at "2700" | |||
| // (cannot easily test gating of crate-level #[no_std]; but non crate-level is below at "2600") | |||
| #![proc_macro_derive()] //~ WARN `#[proc_macro_derive]` only has an effect | |||
| #![proc_macro_derive(Test)] //~ WARN `#[proc_macro_derive]` only has an effect | |||
There was a problem hiding this comment.
Note that a breaking change happened here! The code before this change would now produce an error.
Previously a proc_macro_derive applied to the crate could have any arguments it wants, it was not checked. This PR fixes this bug, and this now errors.
I discussed this privately with @jdonszelmann, and she said this is fine to change, though we can consider doing a crater run for this just to make sure if you wish.
rustbot
added
the
S-waiting-on-review
|
Some changes occurred in compiler/rustc_attr_parsing Some changes occurred in compiler/rustc_passes/src/check_attr.rs Some changes occurred in compiler/rustc_attr_data_structures |
This comment has been minimized.
This comment has been minimized.
|
Some changes occurred in src/tools/clippy cc @rust-lang/clippy |
|
This could use a little squash I think |
|
Not everything, but some commits are a bit small |
JonathanBrouwer
force-pushed
the
proc_macro_attrs
branch
from
a6b7f55 to
432cde9
Compare
|
@jdonszelmann I've squashed things down a bit. I'm experimenting a bit with what commit size is best for reviewability, this was clearly too much of a good thing :P |
JonathanBrouwer
force-pushed
the
proc_macro_attrs
branch
from
432cde9 to
7a60c84
Compare
|
r? @jdonszelmann |
|
|
|
☔ The latest upstream changes (presumably #143645) made this pull request unmergeable. Please resolve the merge conflicts. |
Signed-off-by: Jonathan Brouwer <jonathantbrouwer@gmail.com>
Signed-off-by: Jonathan Brouwer <jonathantbrouwer@gmail.com>
Signed-off-by: Jonathan Brouwer <jonathantbrouwer@gmail.com>
Signed-off-by: Jonathan Brouwer <jonathantbrouwer@gmail.com>
Signed-off-by: Jonathan Brouwer <jonathantbrouwer@gmail.com>
JonathanBrouwer
force-pushed
the
proc_macro_attrs
branch
from
7a60c84 to
a39cacf
Compare
Ports
#[proc_macro],#[proc_macro_attribute],#[proc_macro_derive]and#[rustc_builtin_macro]to the new attribute parsing infrastructure for #131229 (comment)I've split this PR into commits for reviewability, and left some comments to clarify things
I did 4 related attributes in one PR because they share a lot of their code and logic, and doing them separately is kind of annoying as I need to leave both the old and new parsing in place then.
r? @oli-obk
cc @jdonszelmann